Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add explicit break when resolver pool is small #52

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jeteon
Copy link

@jeteon jeteon commented Feb 7, 2017

There is a period during the bootstrapping process (when there are still very few verified servers in resolver_q) and possibly near the end (when the list is exhausted) when a resolver might only have one nameserver. In this case, it will hammer that same resolver query after query with no break. This change puts in an explicit wait for these transient times in order to keep to the implicit contract of not querying the same server more than about once every 5 seconds.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant